Skip to content

[3.7] bpo-37885: venv: Don't produce unbound variable warning on deactivate (GH-15330)#15974

Merged
Yhg1s merged 1 commit into
python:3.7from
miss-islington:backport-5209e58-3.7
Sep 11, 2019
Merged

[3.7] bpo-37885: venv: Don't produce unbound variable warning on deactivate (GH-15330)#15974
Yhg1s merged 1 commit into
python:3.7from
miss-islington:backport-5209e58-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Sep 11, 2019

Copy link
Copy Markdown
Contributor

Before, running deactivate from a bash shell configured to treat undefined variables as errors (set -u) would produce a warning:

$ python3 -m venv test
$ source test/bin/activate
(test) $ deactivate
-bash: $1: unbound variable

(cherry picked from commit 5209e58)

Co-authored-by: Daniel Abrahamsson hamsson@gmail.com

https://bugs.python.org/issue37885

…pythonGH-15330)

Before, running deactivate from a bash shell configured to treat undefined variables as errors (`set -u`) would produce a warning:

```
$ python3 -m venv test
$ source test/bin/activate
(test) $ deactivate
-bash: $1: unbound variable
```
(cherry picked from commit 5209e58)

Co-authored-by: Daniel Abrahamsson <hamsson@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@danabr and @Yhg1s: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@danabr and @Yhg1s: Status check is done, and it's a success ✅ .

@Yhg1s
Yhg1s merged commit d126fbd into python:3.7 Sep 11, 2019
@miss-islington
miss-islington deleted the backport-5209e58-3.7 branch September 11, 2019 15:56
MetricMike added a commit to MetricMike/asdf-awscli that referenced this pull request Feb 9, 2021
    pypa/virtualenv#1342
    python/cpython#15974

    venv deactivate fails with set -euo pipefail
    fixed in 3.7.5+
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants